-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
toc without highlighting and without mobile view #214
Conversation
8e60cfe
to
046272b
Compare
11baefa
to
4c45277
Compare
@@ -5,6 +5,6 @@ export default { | |||
{path: "/sub/two", name: "Two"} | |||
], | |||
toc: { | |||
label: "Contents" | |||
label: "On this page" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want to follow the exact same naming as Vitepress? I think we should opt for something different
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the override test, it needs to be different from the new default which is "Contents"
We discussed fast-follows:
This css rule will show a hamburger next to (or instead of) the toc header: #observablehq-toc summary::before {
content: '☰ ';
} |
Quick and simple TOC without the following:
client.js
here using a doubly linked list to keep track of theprev
,current
, andnext
sections. It could be too slow to react to ascroll
event every single time to determine which section should be highlighted.